Skip to content

Conversation

Abhinavpv28
Copy link
Contributor

No description provided.

@Abhinavpv28 Abhinavpv28 requested a review from a team as a code owner June 2, 2025 07:46
@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Logically dead code

Execution cannot reach this statement: "this->m_statusMutex.lock();".

Medium Impact, CWE-561
DEADCODE

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
MaintenanceManager/MaintenanceManager.cpp:444

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Data race condition

Accessing "this->m_task_map" without holding lock "WPEFramework::Plugin::MaintenanceManager.m_statusMutex". Elsewhere, "WPEFramework::Plugin::MaintenanceManager.m_task_map" is written to with "MaintenanceManager.m_statusMutex" held 11 out of 15 times.

Medium Impact, CWE-366
MISSING_LOCK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
MaintenanceManager/MaintenanceManager.cpp:504

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Logically dead code

Execution cannot reach this statement: "exitOnNoNetwork = true;".

Medium Impact, CWE-561
DEADCODE

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
MaintenanceManager/MaintenanceManager.cpp:439

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Data race condition

Accessing "this->g_task_status" without holding lock "WPEFramework::Plugin::MaintenanceManager.m_statusMutex". Elsewhere, "WPEFramework::Plugin::MaintenanceManager.g_task_status" is written to with "MaintenanceManager.m_statusMutex" held 13 out of 16 times.

Medium Impact, CWE-366
MISSING_LOCK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
MaintenanceManager/MaintenanceManager.cpp:530

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Uninitialized scalar variable

Using uninitialized value "result".

High Impact, CWE-457
UNINIT

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
MaintenanceManager/MaintenanceManager.cpp:1208

@rdkcmf-jenkins
Copy link
Contributor

Coverity Issue - Resource leak

Variable "fp" going out of scope leaks the storage it points to.

High Impact, CWE-404
RESOURCE_LEAK

Issue location

This issue was discovered outside the diff for this Pull Request. You can find it at:
MaintenanceManager/MaintenanceManager.cpp:2085

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants